iOS SDK の RTCAudioSession に pauseRecording()/resumeRecording() 追加 by t-miya · Pull Request #130 · shiguredo-webrtc-build/webrtc-build

t-miya · 2025-12-09T07:42:29Z

iOS SDK での音声ハードミュートのため、pauseRecording()/resumeRecording() を追加します

RTCPeerConnectionFactory

init 時に AudioDeviceModule を受け取るパスを追加しました。
用途として RTCAudioDeviceModule のポインタを保持しライフサイクル管理します。
既存の DeviceModule を受け取るパスも維持しています。

RTCAudioDeviceModule(新規)

AudioDeviceModuleIOS を生成し、pauseRecording()/resumeRecording() するためのラッパーとして今回追加しました。

AudioDeviceModuleIOS

pauseRecording()/resumeRecording() を追加しました。
AudioDeviceIOS の pauseRecording()/resumeRecording() を呼び出します。

AudioDeviceIOS

追加

paused_recording_ フラグを追加して録音停止中であることを管理します。
pauseRecording() では recording_.store(0, std::memory_order_release); で録音フラグを OFF にします。
resumeRecording() では paused_recording_ を false にした上で startRecording() を実行して録音を再開します。

既存の修正

startRecording() はポーズ中 paused_recording_=true で呼ばれた場合は resumeRecording() にフォールスルーするようにしています。
stopRecording() では録音ポーズ中に呼ばれることがあるため paused_recording_ のチェックを追加して、録音停止中でも paused_recording_=true の場合は解放処理を行うようにしています。

melpon · 2025-12-09T11:50:19Z

RTCAudioSession.h に pauseRecording, resumeRecording を生やしているけど、これはオーディオセッションとは何の関係もない
g_last_audio_device_module_ios というグローバル変数を使っているが、複数の PeerConnectionFactory を生成したら破綻する

ので設計を見直した方が良さそうです。

調べた限り ADM は ObjC 側に公開されていないので、

RTCAudioDeviceModule.{h,mm} を新しく作り、そこの初期化時に AudioDeviceModuleIOS を生成して保持しておき、メソッドとして pauseRecording, resumeRecording を生やしておく（本当は ADM が必要とするメソッド全部用意するべきだけど大変過ぎるので最低限だけ）
- 多分 RTCAudioDeviceModule+Native.h みたいなのも必要になるはずなので、適宜作る
RTCPeerConnectionFactory に initWithEncoderFactory:decoderFactory:audioDeviceModule: を生やして、RTCAudioDeviceModule から webrtc::AudioDeviceModule を取得して dependencies.adm に設定する

という感じにすれば RTCAudioDeviceModule の関数を呼んで pauseRecording, resumeRecording が出来そうです。

こうすると、iOS SDK 側の NativePeerChannelFactory.swift#L44-L47 あたりで

    adm = RTCAudioDeviceModule()
    nativeFactory =
      RTCPeerConnectionFactory(
        encoderFactory: encoder,
        decoderFactory: decoder,
        audioDeviceModule: adm)

みたいに書けて、あとは adm を適切に使えば iOS SDK から pause/resume できるようになるはず。

t-miya · 2025-12-10T01:34:46Z

RTCAudioDeviceModule を追加する方向で修正します

melpon · 2025-12-10T17:12:21Z

シングルトンパターン: RTCAudioDeviceModule のシングルトンパターン実装は、単一ADMシナリオを優雅に処理し、前述のグローバル変数の問題を回避しています。

何も回避できてないです。シングルトンはグローバル変数と同じです。

t-miya · 2025-12-11T02:32:53Z

シングルトンを避ける方向で再度見直し中です

melpon · 2025-12-11T10:18:54Z

WeakPtr 関連は無くても平気なはずです。今までは RTCPeerConnectionFactory が解放された時に ADM が解放されてたのが、RTCPeerConnectionFactory と RTCAudioDeviceModule の両方が解放された時に ADM が解放されるようになるだけです。

t-miya · 2025-12-11T10:21:46Z

なるほど。了解です。

…ording() で initRecording() は必要な場合のみ行う

t-miya · 2025-12-12T05:49:19Z

@melpon
WeakPtr を外す対応しました。その他も対応済みのため再度レビューお願いします🙏

melpon · 2025-12-12T06:33:07Z

+                               decoderFactory
+               audioDevice:(nullable id<RTC_OBJC_TYPE(RTCAudioDevice)>)audioDevice
+          audioDeviceModule:
+              (nullable RTC_OBJC_TYPE(RTCAudioDeviceModule)*)audioDeviceModule;


この関数は不要そう

audioDevice、audioDeviceModule 両方を引数に取る版は削除して、元の実装を維持しつつ
audioDeviceModule 引数版を追加するように修正しました

melpon · 2025-12-12T06:33:23Z

+  // RTCAudioDeviceModule は内部でネイティブ ADM を生成・保持する(scoped_refptr による所有)
+  // RTCPeerConnectionFactory は初期化時に RTCAudioDeviceModule から生ポインタを受け取る
+  // RTCAudioDeviceModule が破棄されると内部 ADM も破棄される
+  __weak RTC_OBJC_TYPE(RTCAudioDeviceModule)* _objcAudioDeviceModule;


ここに ADM 保持する意味ないです

_objcAudioDeviceModule; メンバーを削除しました

melpon · 2025-12-12T06:34:23Z

+
+ /* Initialize object with injectable video encoder/decoder factories and
+- * injectable ADM */
+ * injectable ADM (RTCAudioDevice) */


ここの変更は不要そう

変更を元に戻しました

melpon · 2025-12-12T06:37:18Z

+- (instancetype)init;
+
+// ネイティブ ADM を取得する(所有権移動なし、未初期化なら nullptr)
+- (void *)getNativeAudioDeviceModule;


このメソッドは iOS SDK に公開されるヘッダーなんですが、

正しいポインタに戻す方法が無い

そもそも公開する側で使うメソッドではない

ので使い道が無いです。RTCAudioDeviceModule+Private.h とかそういうファイルを作って、ちゃんと型を明示して C++ から参照できる形にして下さい

RTCAudioDeviceModule+Private.h を作成し、nativeAudioDeviceModule 定義を移動しました

melpon · 2025-12-12T06:37:47Z

+// 内部で AudioDeviceModuleIOS を生成する
+- (instancetype)init;
+
+// ネイティブ ADM を取得する(所有権移動なし、未初期化なら nullptr)


コンストラクタで ADM を生成するので未初期化状態にはならないはずです。

null チェック含めて修正しました

melpon · 2025-12-12T06:39:58Z

+- (BOOL)pauseRecording {
+  auto ptr =
+      static_cast<webrtc::ios_adm::AudioDeviceModuleIOS*>(_adm_owner.get());
+  if (!ptr) {


nullptr にはならないはずです

このチェックを外しました

melpon · 2025-12-12T06:44:59Z

+  if (!ptr) {
+    return NO;
+  }
+  return ptr->PauseRecording() == 0;


このクラスは webrtc::AudioDeviceModule の ObjC 版なので、変換をせずにそのまま整数で返すべきです

呼び出しそのままの値を返すようにしました。関数返り値型を NSInteger に変更しました

melpon · 2025-12-12T06:50:47Z

+  // InitRecording に失敗した場合は pause 状態の解除も recording_ の再立て直しも行えないため
+  // 上位にエラーを返す
+  if (!audio_is_initialized_) {
+    if (InitRecording() != 0) {


Resume 時にまだ Init してない場合に Init してあげる必要ありますかね？ちょっとどういう使い方を想定してるのか分からないです。

ここではフラグ操作のみにしました

melpon · 2025-12-12T06:58:10Z

+   return 0;
+ }
+
+int32_t AudioDeviceIOS::PauseRecording() {


PauseRecording/ResumeRecording って、StartRecording/StopRecording と何が違うんでしょうか。見てる感じ StartRecording と StopRecording だけで事足りるように見えます

StopRecording は内部で ShutdownPlayOrRecord() によりデバイス解放する部分が異なっています

ResumeRecording() は直接 StartRecording() を呼んでよいかもしれないです。確認します

AudioDeviceIOS 内の PauseRecording/ResumeRecording は recording_ フラグの操作のみを行い、
startRecording() 等を呼ばないように修正しました

melpon · 2025-12-12T07:00:33Z

+ }
+
+-AudioDeviceModuleIOS::~AudioDeviceModuleIOS() {
+int32_t AudioDeviceModuleIOS::AttachAudioBuffer() {


このあたりどうなってるんだろ、不要な変更に見えます

関数の順番だけ入れ替わった不要な変更になっていたので戻しました

…の操作のみする

…oDeviceModule版それぞれに分離

t-miya · 2025-12-15T03:41:55Z

@melpon
指摘いただいた箇所修正しましたのでレビューお願いします🙏

melpon · 2025-12-15T03:49:36Z

+   }
+
+-  rtc_library("core_audio_helpers_objc") {
+rtc_library("core_audio_helpers_objc") {


インデントが消えてる？

ずれてしまっていたので修正しました

melpon · 2025-12-15T03:55:16Z

+                    (nullable id<RTC_OBJC_TYPE(RTCAudioDevice)>)audioDevice;
+
+/* Initialize object with injectable video encoder/decoder factories and
+ * RTCAudioDeviceModule (pause/resume 用) */


これ Javadoc 形式でも無いし、普通の C コメントでもない中途半端な書き方なので統一して欲しいです。あと日本語で書いて下さい。

コメント形式を直して日本語で書き直しました

melpon · 2025-12-15T03:57:50Z

+   dependencies.env = webrtc::CreateEnvironment();
+-#ifdef WEBRTC_IOS
+-  dependencies.adm = webrtc::CreateAudioDeviceModule(*dependencies.env);
+-#endif


これは消す必要無さそうに見えます

消す必要なかったですね。戻しました

melpon · 2025-12-15T03:58:57Z

+    dependencies.video_decoder_factory =
+        webrtc::ObjCToNativeVideoDecoderFactory(decoderFactory);
+  }
+#ifdef WEBRTC_IOS


iOS でしかパッチを当てないので ifdef は不要そう

そもそもパッチターゲットがあるのを失念してました

RTCAudioDeviceModule を引数に取る initWithEncoderFactory では #ifdef WEBRTC_IOS を削除しました

melpon · 2025-12-15T03:59:44Z

+#ifdef WEBRTC_IOS
+  if (audioDeviceModule) {
+    auto adm_ptr = [audioDeviceModule nativeAudioDeviceModule];
+    if (adm_ptr) {


adm_ptr が nullptr になることは無いはずです

その通りなので adm_ptr の null チェックは外しました

melpon · 2025-12-15T04:10:43Z

+- (instancetype)init;
+
+// 録音を一時停止/再開する
+// 内部で AudioDeviceModuleIOS -> AudioDeviceIOS の pauseRecording()/resumeRecording() を呼び出す


内部のことについて書くのはまあ良いとしても、AudioDeviceModuleIOS が AudioDeviceIOS の関数を呼んでいるかどうかに関してはこのクラスは知らない話なので書くべきではないです。

AudioDeviceIOS 部分のコメントを削除して、AudioDeviceModuleIOS の pauseRecording() を呼ぶ、までにしました
また resumeRecording() とそれぞれコメントを分離しました

melpon · 2025-12-15T04:11:50Z

+#import "RTCAudioDeviceModule.h"
+#include <objc/NSObjCRuntime.h>
+
+#include <arm_acle.h>


このヘッダー何に使ってるんだろ。不要に見えます。

不要でした。削除しました

melpon · 2025-12-15T04:13:02Z

+  return self;
+}
+
+#ifdef __cplusplus


.mm ファイルなのでこの #ifdef いらないです

削除しました

melpon · 2025-12-15T04:15:49Z

+   if (!playing_.load()) {
+     ShutdownPlayOrRecord();
+   }
+  if (recording_.load()) {


この分岐必要なさそう

録音フラグ操作が二重で走っても問題ないので不要でした

melpon · 2025-12-15T04:18:25Z

+     return 0;
+   }
+-  audio_device_buffer_.get()->StartRecording();
+  if (audio_device_buffer_) {


このパッチで audio_device_buffer_ が nullptr になる可能性が増えてるようには見えないので修正する必要無さそうに見えます。他の分岐も同様です。

startRecording()/stopRecording() 含めて audio_device_buffer_ の null チェックを外しました
また関数の呼び出しを audio_device_buffer_.get()-> で統一しました

t-miya · 2025-12-16T04:35:52Z

+ @class RTC_OBJC_TYPE(RTCVideoSource);
+ @class RTC_OBJC_TYPE(RTCVideoTrack);
+ @class RTC_OBJC_TYPE(RTCPeerConnectionFactoryOptions);
+@class RTC_OBJC_TYPE(RTCAudioDeviceModule);


@protocol~ の下に置いていたのを @class~ の並びの方に移動しました

t-miya · 2025-12-17T02:12:49Z

+       sources += [
+         "objc/helpers/UIDevice+RTCDevice.h",
+         "objc/helpers/UIDevice+RTCDevice.mm",
+        "objc/components/audio/RTCAudioDeviceModule.mm",


現状では audio components 関連は RTCAudioDeviceModule のみのため新たにターゲットは切らずここに置く

t-miya · 2025-12-17T03:28:42Z

@melpon
再度修正しましたのでご確認お願いします🙏

melpon · 2025-12-17T03:59:46Z


+- 2025-12-12 [ADD] iOS SDK に RTCAudioDeviceModule を追加する
+  - iOS 実機のマイクインジケータが消灯状態のミュートをできるようにする
+  - RTCPeerConnectionFactory に RTCAudioDeviceModule を引数とする initWithEncoderFactory() を追加する


ObjC の場合これは initWithEncoderFactory:decoderFactory:audioDeviceModule: というメソッド名になります。
引数の名前まで含めてメソッド名で、C++ みたいな引数の型によるオーバーロードは存在しないです。

initWithEncoderFactory:decoderFactory:audioDeviceModule: に修正しました。
その他コメント中で pauseRecording() のように書いていた箇所を pauseRecording 等に修正しました。

melpon

よさそう

t-miya · 2025-12-17T05:41:28Z

ありがとうございました🙇

This comment was marked as abuse.

Sign in to view

This comment was marked as resolved.

Sign in to view

This comment was marked as abuse.

Sign in to view

This comment was marked as resolved.

Sign in to view

t-miya requested a review from melpon December 9, 2025 09:06

This comment was marked as resolved.

Sign in to view

This comment was marked as abuse.

Sign in to view

This comment was marked as resolved.

Sign in to view

t-miya added 8 commits December 12, 2025 13:22

iOS SDK の RTCAudioSession に pauseRecording()/resumeRecording() 追加

57998be

pause_recording 中の stopRecording() 処理を修正、dispose を修正

41600a3

recording_.load() 引数を修正

4a155cd

startRecording() がポーズ中に呼ばれた場合は resumeRecording() にフォールスルーする、resumeRec…

6324529

…ording() で initRecording() は必要な場合のみ行う

resumeRecording() 内で startRecording() に失敗した場合 pause 状態に戻す

559965f

公開APIを RTCAudioSession から RTCAudioDeviceModule に移行

a184f8c

RTCAudioModuleDevice を RTCPeerConnectionFactory で管理するよう修正

5ed4205

RTCAudioDeviceModule の weak_ptr を外す

9359604

t-miya force-pushed the feature/ios-audio-pause-resume branch from cda9dea to 9359604 Compare December 12, 2025 04:24

This comment was marked as resolved.

Sign in to view

melpon reviewed Dec 12, 2025

View reviewed changes

t-miya added 4 commits December 12, 2025 18:27

AudioDeviceIOS 内の pauseRecording()/resumeRecording() は recording_ フラグ…

01df8d1

…の操作のみする

RTCPeerConnectionFactory の initWithEncoderFactory を audioDevice版、audi…

4bd7c15

…oDeviceModule版それぞれに分離

RTCAudioDeviceModule+Private.h を追加、その他関数の整理

d5d0db9

不要なコメント削除

8cc1829

melpon reviewed Dec 15, 2025

View reviewed changes

不要な分岐、ifdef の整理等

4506f17

t-miya commented Dec 16, 2025

View reviewed changes

t-miya added 2 commits December 16, 2025 13:39

改行差分を除去

61f6963

CHANGES 修正

bbddaa4

t-miya commented Dec 17, 2025

View reviewed changes

コメント修正

18b7e38

melpon reviewed Dec 17, 2025

View reviewed changes

t-miya added 2 commits December 17, 2025 13:20

initWithEncoderFactory に関するコメントを ObjC の形式に修正

05cb368

コメント中の関数に関する標記を ObjC 形式に修正

c80883e

melpon approved these changes Dec 17, 2025

View reviewed changes

t-miya merged commit 857a55f into feature/m143.7499 Dec 17, 2025
15 checks passed

t-miya deleted the feature/ios-audio-pause-resume branch December 17, 2025 05:51

t-miya mentioned this pull request Dec 24, 2025

iOS SDK 向けの音声ミュート処理を修正する #131

Merged

Uh oh!

Conversation

t-miya commented Dec 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

RTCPeerConnectionFactory

RTCAudioDeviceModule(新規)

AudioDeviceModuleIOS

AudioDeviceIOS

追加

既存の修正

Uh oh!

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as abuse.

This comment was marked as resolved.

melpon commented Dec 9, 2025

Uh oh!

t-miya commented Dec 10, 2025

Uh oh!

This comment was marked as resolved.

This comment was marked as abuse.

This comment was marked as resolved.

This comment was marked as resolved.

melpon commented Dec 10, 2025

Uh oh!

t-miya commented Dec 11, 2025

Uh oh!

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

melpon commented Dec 11, 2025

Uh oh!

t-miya commented Dec 11, 2025

Uh oh!

This comment was marked as resolved.

This comment was marked as resolved.

t-miya commented Dec 12, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

t-miya commented Dec 9, 2025 •

edited

Loading